Skip to content

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Jan 16, 2025

Signed-off-by: David Karlsson [email protected]

@netlify
Copy link

netlify bot commented Jan 16, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit d7d5762
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/679746ecbc885a000861ca19
😎 Deploy Preview https://deploy-preview-21838--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dvdksn dvdksn force-pushed the sunset-products branch 2 times, most recently from dca4ce6 to 6c37bd0 Compare January 16, 2025 13:54
@dvdksn dvdksn marked this pull request as ready for review January 16, 2025 14:14
@dvdksn dvdksn requested review from a team, justincormack and sflondonla January 16, 2025 14:14
Copy link
Contributor

@sarahsanders-docker sarahsanders-docker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two little nits, but otherwise I really like this approach. one page for deprecated/retired features is much easier to find and digest

Docker Toolbox was used on older systems where Docker Desktop could not run. It
bundled Docker Machine, Docker Engine, and Docker Compose into a single
installer. Toolbox is no longer maintained and is effectively replaced by
Docker Desktop on current systems. References to Docker Toolbox occasionally
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a link to DD

Copy link
Member

@usha-mandya usha-mandya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dvdksn dvdksn merged commit ca93b58 into docker:main Jan 27, 2025
14 checks passed
@dvdksn dvdksn deleted the sunset-products branch January 27, 2025 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants